Skip to content

build(deps): update test Jetty to 9.4.58#785

Open
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-test-jetty
Open

build(deps): update test Jetty to 9.4.58#785
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-test-jetty

Conversation

@jbeckwith-oai

@jbeckwith-oai jbeckwith-oai commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • align WireMock 2.35.2's Jetty test runtime on the Jetty 9.4 BOM at 9.4.58.v20250814
  • apply the alignment only to the core and OkHttp test configurations
  • keep WireMock 2.35.2, the Java 8 baseline, and both Jackson 2.14.0/2.18.9 compatibility lines unchanged

Security impact

The BOM upgrades the complete org.eclipse.jetty and org.eclipse.jetty.http2 test graph from 9.4.49.v20220914 to 9.4.58.v20250814. This addresses Dependabot alerts #12-17, #24-26, #31, #33, #40, and #49.

Three Jetty alerts cannot be fixed on the Java-8-compatible 9.4 line today:

Those three alerts remain visible rather than being hidden behind a breaking major upgrade.

Compatibility

  • The change stays within Jetty 9.4, and all WireMock-backed tests pass with WireMock 2.35.2.
  • Representative classes from every affected Jetty and HTTP/2 artifact are Java 8 bytecode (major version 52).
  • Core and OkHttp WireMock tests pass with both the Jackson 2.14.0 compatibility floor and the published Jackson 2.18.9 line.
  • The generated Maven POMs for both modules contain no Jetty BOM, Jetty artifact, or WireMock dependency, so downstream dependency resolution and public APIs are unchanged.

Validation

  • ./scripts/lint
  • ./scripts/build
  • ./scripts/test
  • ./scripts/detect-breaking-changes origin/main
  • Jetty dependencyInsight for both test runtime classpaths
  • targeted WireMock suites under Jackson 2.14.0 and 2.18.9 in both modules
  • generated Maven POM inspection for core and OkHttp

@jbeckwith-oai jbeckwith-oai requested a review from a team as a code owner July 15, 2026 22:25
@openai-sdks

openai-sdks Bot commented Jul 15, 2026 •
edited
Loading

Copy link
Copy Markdown

OkTest Summary

✅ 237/237 SDK tests passed in 16.517s for Java SDK PR #785.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 208ms
tests/chat-completions-create.test.ts ✅ Passed 903ms
tests/chat-completions-stream.test.ts ✅ Passed 366ms
tests/files-content-binary.test.ts ✅ Passed 282ms
tests/files-create-multipart.test.ts ✅ Passed 262ms
tests/files-list-pagination.test.ts ✅ Passed 332ms
tests/initialize-config.test.ts ✅ Passed 167ms
tests/instance-isolation.test.ts ✅ Passed 195ms
tests/models-list.test.ts ✅ Passed 171ms
tests/responses-background-lifecycle.test.ts ✅ Passed 281ms
tests/responses-body-method-errors.test.ts ✅ Passed 484ms
tests/responses-cancel-timeout.test.ts ✅ Passed 212ms
tests/responses-cancel.test.ts ✅ Passed 311ms
tests/responses-compact-retries.test.ts ✅ Passed 549ms
tests/responses-compact.test.ts ✅ Passed 446ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 341ms
tests/responses-create-advanced.test.ts ✅ Passed 1.434s
tests/responses-create-disconnect.test.ts ✅ Passed 1.15s
tests/responses-create-errors.test.ts ✅ Passed 330ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 330ms
tests/responses-create-retries.test.ts ✅ Passed 237ms
tests/responses-create-stream-failures.test.ts ✅ Passed 251ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 235ms
tests/responses-create-stream-wire.test.ts ✅ Passed 5.959s
tests/responses-create-stream.test.ts ✅ Passed 434ms
tests/responses-create-terminal-states.test.ts ✅ Passed 371ms
tests/responses-create-timeout.test.ts ✅ Passed 228ms
tests/responses-create.test.ts ✅ Passed 284ms
tests/responses-delete.test.ts ✅ Passed 293ms
tests/responses-input-items-errors.test.ts ✅ Passed 308ms
tests/responses-input-items-list.test.ts ✅ Passed 314ms
tests/responses-input-items-options.test.ts ✅ Passed 200ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 279ms
tests/responses-input-tokens-count.test.ts ✅ Passed 366ms
tests/responses-malformed-inputs.test.ts ✅ Passed 4.722s
tests/responses-not-found-errors.test.ts ✅ Passed 411ms
tests/responses-parse.test.ts ✅ Passed 587ms
tests/responses-retrieve-retries.test.ts ✅ Passed 1.087s
tests/responses-retrieve.test.ts ✅ Passed 325ms
tests/responses-stored-method-errors.test.ts ✅ Passed 1.098s
tests/retry-behavior.test.ts ✅ Passed 3.573s
tests/sdk-error-shape.test.ts ✅ Passed 498ms

View OkTest run #29462769641

SDK merge (26b718a854c1) · head (46d8db4bc528) · base (845b2bc87fb0) · OkTest (fd7a761c81c1)

@jbeckwith-oai jbeckwith-oai force-pushed the codex/update-test-jetty branch from c306d8f to 46d8db4 Compare July 16, 2026 00:55
@jbeckwith-oai jbeckwith-oai changed the title fix(deps): update test Jetty to 9.4.58 build(deps): update test Jetty to 9.4.58 Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jbeckwith-oai